Skip to content

chore(github): make repo agent-ready (fix AGENTS.md, add PR/issue templates) - #396

Merged
looztra merged 2 commits into
mainfrom
chore/agent-ready
Sep 6, 2026
Merged

chore(github): make repo agent-ready (fix AGENTS.md, add PR/issue templates)#396
looztra merged 2 commits into
mainfrom
chore/agent-ready

Conversation

@looztra

@looztra looztra commented Sep 6, 2026

Copy link
Copy Markdown
Owner

Pull Request

Description

Fixes AI-readiness gaps found while auditing this repo for agent-friendliness, skipping Copilot-specific assets and MCP config per request.

Changes

  • AGENTS.md was a verbatim copy of the yamkix repo's agent instructions (wrong repo name/URL, and src/yamkix/* paths that don't exist here). Fixed to reference kubesplit and the real src/kubesplit/* modules (config.py, _cli.py, convert.py, k8s_descriptor.py, namespaces.py, output.py), and clarified that YAML formatting is delegated to the external yamkix dependency.
  • Replaced the legacy single-file .github/ISSUE_TEMPLATE.md with structured YAML issue forms (bug_report.yml, feature_request.yml) plus config.yml (blank issues disabled).
  • Added .github/PULL_REQUEST_TEMPLATE.md (none existed before) with a checklist derived from the project's actual poe/make targets.

How to test

pre-commit run --all-files
poe lint:all
poe test

Checklist

  • PR title follows Conventional Commits with a scope
  • poe style run (formatting)
  • poe lint:all passes (ruff, pylint, pyright, ty)
  • poe test (unit tests) passes
  • pre-commit run --all-files passes
  • make integration-tests — not run (no code changes, docs/config only)

AGENTS.md was a verbatim copy of the yamkix repo's agent instructions:
wrong repo name/URL and nonexistent src/yamkix/* paths. Point it at the
actual kubesplit repo and src/kubesplit/* modules.
Replace the legacy single-file .github/ISSUE_TEMPLATE.md with
structured YAML issue forms (bug report, feature request) plus a
disabled blank-issues config, and add a PULL_REQUEST_TEMPLATE.md with
a test-command and checklist derived from the project's own poe/make
targets. No pull request template previously existed.
@codecov

codecov Bot commented Sep 6, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.14%. Comparing base (ab63e2d) to head (e6ad112).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #396   +/-   ##
=======================================
  Coverage   79.14%   79.14%           
=======================================
  Files          11       11           
  Lines         235      235           
  Branches       28       28           
=======================================
  Hits          186      186           
  Misses         45       45           
  Partials        4        4           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

@looztra
looztra enabled auto-merge (squash) September 6, 2026 13:43
@looztra
looztra disabled auto-merge September 6, 2026 13:52
@looztra
looztra merged commit 305a8e5 into main Sep 6, 2026
12 checks passed
@looztra
looztra deleted the chore/agent-ready branch September 6, 2026 13:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant